From: Jasper St. Pierre <jstpierre@mecheye.net>
Date: Fri, 21 Jun 2013 19:24:50 +0000 (-0400)
Subject: messageTray: Ensure notifications are focused after they are expanded
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~4026
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=aa7ed319e96105984cac7d594d41ec1e35a7fee6;p=gnome-shell.git

messageTray: Ensure notifications are focused after they are expanded

If we focus notifications before they're expanded, the body and action
area won't be visible, and the can_focus members like the text entry
will not be able to be focused.

Ensure that all of the all actors that would be in an expanded notification
are visible before we attempt to focus them.

https://bugzilla.gnome.org/show_bug.cgi?id=698778
---

diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index c0569247a..afbcf269a 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -2597,16 +2597,16 @@ const MessageTray = new Lang.Class({
     },
 
     _expandNotification: function(autoExpanding) {
-        // Don't focus notifications that are auto-expanding.
-        if (!autoExpanding)
-            this._ensureNotificationFocused();
-
         if (!this._notificationExpandedId)
             this._notificationExpandedId =
                 this._notification.connect('expanded',
                                            Lang.bind(this, this._onNotificationExpanded));
         // Don't animate changes in notifications that are auto-expanding.
         this._notification.expand(!autoExpanding);
+
+        // Don't focus notifications that are auto-expanding.
+        if (!autoExpanding)
+            this._ensureNotificationFocused();
     },
 
     _onNotificationExpanded: function() {